aitools: add skills command for coding agents#4125
Merged
Conversation
Collaborator
|
Commit: e7aa354
25 interesting tests: 8 RECOVERED, 5 KNOWN, 5 SKIP, 5 flaky, 2 FAIL
Top 50 slowest tests (at least 2 minutes):
|
4a6039e to
9448fc0
Compare
Add `databricks experimental aitools skills` subcommand: - `skills list` - list available skills - `skills install` - install all skills to ~/.claude/skills/ Includes databricks-apps skill with reference docs synced from the appkit template.
1794780 to
9550bfa
Compare
9550bfa to
b6254d8
Compare
b6254d8 to
67a2c80
Compare
67a2c80 to
e865a15
Compare
Skills now fetched from databricks/databricks-agent-skills repo.
e865a15 to
ef1b64b
Compare
- Support 8 agents: Claude Code, Cursor, Windsurf, Cline, Roo Code, Codex CLI, Amp, OpenCode - Auto-detect installed agents and print which were found - Use symlinks when multiple agents detected (canonical location: ~/.databricks/agent-skills/) - Fallback to copy if symlink fails (Windows without admin) - Support GitHub token from env vars or gh CLI for private repo access Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
5591161 to
882c772
Compare
- Create agents.Agent struct with detection, skills dir, and optional MCP install - Supported agents: Claude Code, Cursor, Codex CLI, OpenCode, GitHub Copilot, Antigravity - Update skills.go and install.go to use shared registry - Remove duplicated detection logic from claude.go and cursor.go Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
882c772 to
a6dcea6
Compare
keugenek
approved these changes
Jan 23, 2026
lennartkats-db
requested changes
Jan 26, 2026
Comment on lines
+347
to
+350
| useSymlinks := len(detectedAgents) > 1 | ||
| var canonicalDir string | ||
|
|
||
| if useSymlinks { |
Contributor
There was a problem hiding this comment.
Are you always relying on symlinks here? We should have some fallback for Windows
| ) | ||
|
|
||
| // Agent defines a coding agent that can have skills installed and optionally MCP server. | ||
| type Agent struct { |
Contributor
There was a problem hiding this comment.
Great to have this interface!!
experimental/aitools/cmd/skills.go
Outdated
| if len(detectedAgents) == 0 { | ||
| cmdio.LogString(ctx, color.YellowString("No supported coding agents detected.")) | ||
| cmdio.LogString(ctx, "") | ||
| cmdio.LogString(ctx, "Supported agents: Claude Code, Cursor, Codex CLI, OpenCode, GitHub Copilot, Antigravity") |
| } | ||
|
|
||
| // getHomeDir returns home directory, handling Windows USERPROFILE. | ||
| func getHomeDir() (string, error) { |
Contributor
There was a problem hiding this comment.
There should be an existing function like this already
lennartkats-db
approved these changes
Jan 27, 2026
Collaborator
|
Commit: e450099
25 interesting tests: 8 RECOVERED, 7 flaky, 5 KNOWN, 5 SKIP
Top 50 slowest tests (at least 2 minutes):
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Add
databricks experimental aitools skillssubcommand to install Agent Skills for coding agents:skills list- list available skills from the databricks-agent-skills reposkills install [skill-name]- install skills to detected coding agentsMulti-agent support
Automatically detects and installs skills for:
~/.claude/skills/)~/.cursor/skills/)~/.codex/skills/)~/.config/opencode/skills/)~/.copilot/skills/)~/.gemini/antigravity/global_skills/)When multiple agents are detected, skills are installed to a canonical location (
~/.databricks/agent-skills/) and symlinked to each agent to avoid duplication.Refactored agent registry
Unified agent detection logic in
lib/agents/agents.go- shared between skills installation and MCP installation commands.Why
Skills are simpler than MCP for providing domain-specific guidance to AI agents. While MCP requires a running server, skills are static documentation that agents load on demand. This gives users a lightweight way to get Databricks-specific guidance across all major coding agents.
Tests
Manual testing: